Skip to content

[PWGCF] Femto: Allow multiple registries for SH histograms - #17296

Closed
IgorPtak wants to merge 1 commit into
AliceO2Group:masterfrom
IgorPtak:spherical-harmonics-registry
Closed

[PWGCF] Femto: Allow multiple registries for SH histograms#17296
IgorPtak wants to merge 1 commit into
AliceO2Group:masterfrom
IgorPtak:spherical-harmonics-registry

Conversation

@IgorPtak

@IgorPtak IgorPtak commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Problem

SH histograms scale as nCent * nKt * (2 * nJM + 2) and overflow the
512-histogram limit of a single HistogramRegistry.

Solution

PairHistManager::init() takes an optional pool of registries for SH
histograms and assigns each (cent, kT) cell to one with room. Builders
forward separate pools for same-event and mixed-event.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@vkucera

vkucera commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

If you need more than 512 histograms, I think you are doing something wrong.

@alibuild

alibuild commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for be6bd63 at 2026-08-05 10:23:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for be6bd63 at 2026-08-05 09:58:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for be6bd63 at 2026-08-05 12:53:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@IgorPtak
IgorPtak marked this pull request as draft August 5, 2026 13:24
@IgorPtak IgorPtak closed this Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants